Skip to content

Tests for the macOS CI - #1723

Closed
illwieckz wants to merge 6 commits into
masterfrom
illwieckz/test-macos-ci
Closed

Tests for the macOS CI#1723
illwieckz wants to merge 6 commits into
masterfrom
illwieckz/test-macos-ci

Conversation

@illwieckz

Copy link
Copy Markdown
Member

👀

@illwieckz
illwieckz force-pushed the illwieckz/test-macos-ci branch 3 times, most recently from aab8474 to e972d6f Compare August 4, 2025 02:29
@illwieckz

Copy link
Copy Markdown
Member Author

@VReaperV @slipher that's stupid but it worked…

@illwieckz

Copy link
Copy Markdown
Member Author

I've just set both CFLAGS and CXXFLAGS to -Wno-c++14-extensions before calling CMake…

@slipher

slipher commented Aug 4, 2025

Copy link
Copy Markdown
Member

I think it would be better to put this in the CI script than the build system

@illwieckz
illwieckz force-pushed the illwieckz/test-macos-ci branch from 9b388a5 to 18807e8 Compare August 4, 2025 02:46
@illwieckz

Copy link
Copy Markdown
Member Author

Well, both work, so we can do as we prefer.

@illwieckz
illwieckz force-pushed the illwieckz/test-macos-ci branch from 18807e8 to 3afd870 Compare August 4, 2025 02:50
@VReaperV

VReaperV commented Aug 4, 2025

Copy link
Copy Markdown
Contributor

Thanks, I've added the commit to #1718.

@illwieckz illwieckz closed this Aug 4, 2025
@illwieckz
illwieckz deleted the illwieckz/test-macos-ci branch August 4, 2025 14:43
@illwieckz illwieckz changed the title I do tests too… Tests for the macOS CI Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants